-
-
Notifications
You must be signed in to change notification settings - Fork 176
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
allow unix-socket connection in RedisSettings (socket_address) #271
Conversation
I'm wondering how this will interact with #259? I want to get that merged before the next release. |
It will probably need a few minor modifications. (adding a |
#259 is merged, if you still want to work on this, I'd be happy to review an update. |
Nice! |
I think it's ready, as long as we have redis>=4.2.0rc3. |
There's lots of conflicts here, you'll need to rebase/merge with master. |
following the recommendation in #209 (comment)